Skip to content

fix(openapi): nullable default values in operation openapi definition#7321

Merged
soyuka merged 1 commit into
api-platform:4.1from
soyuka:ref/7318
Jul 29, 2025
Merged

fix(openapi): nullable default values in operation openapi definition#7321
soyuka merged 1 commit into
api-platform:4.1from
soyuka:ref/7318

Conversation

@soyuka

@soyuka soyuka commented Jul 29, 2025

Copy link
Copy Markdown
Member

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Don't we need to fix the YamlResourceExtractor too ?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

probably

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Are this changes still needed as the MR #7315 has been merged ?

Otherwise, i realized than i have done partially the job (see: #7322).
If you want to integrate this changes here i can decline the MR.

@soyuka
soyuka merged commit d1abfc0 into api-platform:4.1 Jul 29, 2025
1 check failed
@soyuka

soyuka commented Jul 29, 2025

Copy link
Copy Markdown
Member Author

We need this at least for the schema and I think having defaults is still a nice thing to have.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants